Description

repro https://github.com/brillout/vite-regression-2023-nov-14

We add the ESM proxy guard unconditionally to error when you import something like:

import { non_existent } from './mod'

However, we don't have to add the guard for

import * as mod from './mod'

if (mod.non_existent) // ...

This PR fixes this by checking if they are imported names from the ./mod before adding the guard. In which case, namespace imports would never have imported names.

Other things I changed along the way:

1. Renamed namedImportSpecifiers to importedNames: Matches the AST name and because the array can now include "default" when you do import mod from './mod'. In reality, this is the same as import { default as mod } from './mod' and I noticed Node.js handles this way too.
2. For the emulated Node.js errors, add a [vite] prefix so we can easily identify if it's coming from Vite in the future.

Additional context

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines, especially the Pull Request Guidelines.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Update the corresponding documentation if needed.
• Ideally, include relevant tests that fail without this PR but pass with it.